Skip to content

Changes to github workflows#409

Draft
imatsoup wants to merge 3 commits intomasterfrom
workflows-changes
Draft

Changes to github workflows#409
imatsoup wants to merge 3 commits intomasterfrom
workflows-changes

Conversation

@imatsoup
Copy link
Copy Markdown
Collaborator

About the PR

Made changes we'd been talking about to publish workflows and added changelog.yml (ripped from deltav) workflow file to make changelogs easier.

Why / Balance

I wish to put an end to @arenaconspiracy's suffering

Technical details

Media

Requirements

Breaking changes

Changelog

🆑

  • add: Automated changelog workflows to make changelogs easier to do.

Co-Authored-By: Debug <49997488+debugok@users.noreply.github.com>
@github-actions github-actions bot added S: Untriaged Status: Needs appropriate labels. size/S S: Needs Review Status: This PR requires new reviews before it can be merged. Changes: No C# changes: Might require no knowledge of C#. and removed S: Needs Review Status: This PR requires new reviews before it can be merged. S: Untriaged Status: Needs appropriate labels. size/S labels Sep 26, 2025
@imatsoup
Copy link
Copy Markdown
Collaborator Author

Note: Schema validators failing is unrelated. Should maybe consider turning them off too while those are broken upstream.

@arenaconspiracy arenaconspiracy self-requested a review September 27, 2025 11:22
@github-actions github-actions bot added the S: Needs Review Status: This PR requires new reviews before it can be merged. label Sep 27, 2025
- name: Generate Changelog
run: |
cd "Tools/changelog"
node changelog.js
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is calling a script that we don't actually have, as far as I can tell. See DeltaV-Station/Delta-v#11 having added Tools/Changelogs/changelog.js

types: [closed]

env:
GITHUB_TOKEN: ${{ secrets.BOT_TOKEN }}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have these secrets set up? I'm not 100% on where they're supposed to be stored.

Copy link
Copy Markdown

@arenaconspiracy arenaconspiracy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Accidentally pressed "single comment" instead of starting a review on prior comments, whoops. They're there though.

@arenaconspiracy arenaconspiracy added enhancement New feature or request P1: High Priority: Higher priority than other items, but isn’t an emergency. S: Awaiting Changes Status: Changes are required before another review can happen. labels Sep 27, 2025
@imatsoup imatsoup marked this pull request as draft September 27, 2025 16:52
@imatsoup
Copy link
Copy Markdown
Collaborator Author

Converted to a draft until secrets and vars are worked out

@github-actions github-actions bot added size/M and removed size/S labels Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: No C# changes: Might require no knowledge of C#. enhancement New feature or request P1: High Priority: Higher priority than other items, but isn’t an emergency. S: Awaiting Changes Status: Changes are required before another review can happen. S: Needs Review Status: This PR requires new reviews before it can be merged. size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants